Added a deprecation notice and migration path to the Behat core progress formatter. - #86
Conversation
📝 WalkthroughWalkthroughREADME.md marks the extension deprecated through 31 December 2026 and documents migration to Behat core’s ChangesDeprecation and migration
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related issues
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Line 90: Insert a blank line immediately before the fenced YAML block in the
README section shown, ensuring the opening fence is separated from preceding
content to satisfy markdownlint MD031.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| Upgrade Behat, remove this extension from your `behat.yml`, and enable the option on the built-in `progress` formatter instead: | ||
|
|
||
| >behat.yml | ||
| ```yaml |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Add a blank line before the fenced block.
This is reported by markdownlint as MD031 and may fail documentation checks.
Proposed fix
>behat.yml
+
```yaml📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ```yaml | |
| >behat.yml | |
🧰 Tools
🪛 markdownlint-cli2 (0.23.0)
[warning] 90-90: Fenced code blocks should be surrounded by blank lines
(MD031, blanks-around-fences)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.md` at line 90, Insert a blank line immediately before the fenced YAML
block in the README section shown, ensuring the opening fence is separated from
preceding content to satisfy markdownlint MD031.
Source: Linters/SAST tools
Summary
Behat's built-in
progressformatter just picked up aninline_failuresoption, which is the same behavior this package has provided since day one. This adds a deprecation warning to the top of the README and a new "Migrating to Behat core" section that walks through the switch, since the reason this extension existed is now available upstream.Changes
[!WARNING]banner above the intro that flags the package as deprecated, notes the support window through 31 December 2026, links to the upstream Behat issue that prompted the feature, and points to the migration section.behat.ymlconfig and command-line equivalent for enablinginline_failureson core'sprogressformatter, plus thecomposer removestep to drop this package.show_outputkeeps the same four values (yes,no,on-fail,in-summary) in core, so that part of abehat.ymlcarries over unchanged.inline_failuresis merged into Behat's3.xbranch but hasn't shipped in a tagged release yet (v3.32.0is still the latest), so migration has to wait for the next release.Before / After
Summary by CodeRabbit